Hi,

Thanks for the contribution!
To make the review easier, especially for those who don’t read the code in detail, could you please add before-and-after screenshots for the affected views?

Hello,

First, I added the feature to add IOB.
Then, I added the option to choose where to group the next value.
Finally, I added colors, inspired by Trio.

Here are screenshots:

Hi @luborjurena, and thank you for your contribution. A few comments and questions, let me know if I misunderstood anything.

1. ContactColorMode.parseBGValue() is dead code
   parseBGValue(_:) in ContactColorMode.swift:81-87 is never called anywhere in this PR or called from textColor(for:).
   Not sure if this is code that should be used somewhere?

2. Dynamic color applies to ALL contact types, not just BG
   textColor() in ContactImageUpdater uses the dynamic color for every contact (Trend, Delta, IOB), not just BG. A separate IOB contact showing "2.5U" will be colored green/yellow/red based on the BG value, which may confuse users. Consider whether IOB/Delta/Trend contacts should always use the static color?

3. Potential double-update on device status
   In DeviceStatus.swift:302-310, a contact update is triggered when IOB is enabled. But BGData.swift already triggers a contact update on every BG reading. If a device status update arrives shortly after a BG update, you get two rapid contact writes. Consider debouncing or checking if the IOB value actually changed before triggering, this could cause delays but doubling (or more) the amounts of updates to the contacts might be problematic.

4. iobText default is empty string, others have defaults
   Observable.swift — iobText defaults to "" while directionText defaults to "-" and deltaText to "+0". If a contact update fires before device status loads, the IOB field will render as blank. Consider defaulting to "--" for consistency.

5. Minor: stale strikethrough only on BG type before, now on all
   The old code only applied strikethroughStyle to the BG contact type. The new code applies it to all contact types, was this intentional? We could have a stale bg while IOB is up to date?